-
Notifications
You must be signed in to change notification settings - Fork 15k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: re-enable the spellchecker when new language list set #26119
Conversation
@@ -2,9 +2,19 @@ import * as childProcess from 'child_process'; | |||
import * as path from 'path'; | |||
import * as http from 'http'; | |||
import * as v8 from 'v8'; | |||
import { SuiteFunction, TestFunction } from 'mocha'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This stuff I just added because it was annoying me, you can now do ifdescribe().only
Chromium recently added prefs logic to disable the spellchecker if the list of languages is empty, but the logic to re-enable if the languages are provided again lives in another part of Chromium. This change makes it so our API re-enables the spellchecker correctly when required.
44bebb8
to
7528f2f
Compare
Release Notes Persisted
|
I was unable to backport this PR to "11-x-y" cleanly; |
* fix: re-enable the spellchecker when new language list set Chromium recently added prefs logic to disable the spellchecker if the list of languages is empty, but the logic to re-enable if the languages are provided again lives in another part of Chromium. This change makes it so our API re-enables the spellchecker correctly when required. * chore: fix lint
@MarshallOfSound has manually backported this PR to "11-x-y", please check out #26129 |
* fix: re-enable the spellchecker when new language list set Chromium recently added prefs logic to disable the spellchecker if the list of languages is empty, but the logic to re-enable if the languages are provided again lives in another part of Chromium. This change makes it so our API re-enables the spellchecker correctly when required. * chore: fix lint
…26129) * fix: re-enable the spellchecker when new language list set Chromium recently added prefs logic to disable the spellchecker if the list of languages is empty, but the logic to re-enable if the languages are provided again lives in another part of Chromium. This change makes it so our API re-enables the spellchecker correctly when required. * chore: fix lint
Chromium recently added prefs logic to disable the spellchecker if the list of languages is empty, but the logic to re-enable if the languages are provided again lives in another part of Chromium. This change makes it so our API re-enables the spellchecker correctly when required.
Refs: https://chromium-review.googlesource.com/c/chromium/src/+/2390300
Notes: Fixed issue where setting the spellchecker languages to an empty array would permanently disable the spellchecker till the end of time